Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore/check events from logs #448

Merged
merged 3 commits into from
Oct 4, 2024

Conversation

maharifu
Copy link
Contributor

@maharifu maharifu commented Oct 4, 2024

Related Github tickets

Background

If we get fake/malicious events on skyway, we would get pigeons stuck trying to find them on the external chains.
This PR doesn't completely solve the issue, but at least we don't get stuck now.

Also updated paloma dependency and README for new release.

Testing completed

  • test coverage exists or has been added/updated
  • tested in a private testnet

Breaking changes

  • I have checked my code for breaking changes

If we get skyway event blocks from paloma that don't really have valid
events, we fallback to searching events in the latest blocks. This way
if a wrong or malicious event falls into the skyway module, we still
process new events.
@maharifu maharifu requested a review from taariq October 4, 2024 14:43
Copy link
Contributor

@taariq taariq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LFG

@taariq taariq merged commit 593755e into palomachain:master Oct 4, 2024
2 checks passed
@taariq taariq deleted the chore/check_events_from_logs branch October 4, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Skyway] Paloma needs to identify if an event is real or not and skip otherwise
2 participants